Skip to content

TST: add _honors_copy_keyword to extension test base class#65179

Merged
mroeschke merged 1 commit into
pandas-dev:mainfrom
jbrockmendel:tst-63040
Apr 18, 2026
Merged

TST: add _honors_copy_keyword to extension test base class#65179
mroeschke merged 1 commit into
pandas-dev:mainfrom
jbrockmendel:tst-63040

Conversation

@jbrockmendel
Copy link
Copy Markdown
Member

@jbrockmendel jbrockmendel commented Apr 11, 2026

Summary

  • Add a _honors_copy_keyword method to BaseMissingTests so test_fillna_readonly skips the copy=False assertion for EAs that always return new data regardless. Removes full test overrides in sparse, arrow, string, interval, and decimal test classes.

closes #63040

Test plan

  • All extension test suites pass (sparse, arrow, string, interval, decimal, masked, numpy, datetime, period, json)

🤖 Generated with Claude Code

@jbrockmendel jbrockmendel added the Testing pandas testing functions or related to the test suite label Apr 11, 2026
Add a `_honors_copy_keyword` method to `BaseMissingTests` so that
`test_fillna_readonly` can skip the `copy=False` assertion for EAs
that always return new data regardless. This removes full test
overrides in sparse, arrow, string, interval, and decimal test
classes.

closes pandas-dev#63040

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jbrockmendel jbrockmendel marked this pull request as ready for review April 17, 2026 19:00
@mroeschke mroeschke added this to the 3.1 milestone Apr 18, 2026
@mroeschke mroeschke merged commit 983a3ed into pandas-dev:main Apr 18, 2026
45 checks passed
@mroeschke
Copy link
Copy Markdown
Member

Thanks @jbrockmendel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Testing pandas testing functions or related to the test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

not necessarily for this PR, but should we add an EA attribute specifying whether copy=False is ignored? that way we could avoid overriding tests?

2 participants